Throwing an exception already aborts a function, so there's no need
for a return statement in that case.
Spotted by eslint.
https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/375
getItemPage(item) {
let children = this._getVisibleChildren();
let index = children.indexOf(item);
- if (index == -1) {
+ if (index == -1)
throw new Error('Item not found.');
- return 0;
- }
return Math.floor(index / this._childrenPerPage);
}
openExtraSpace(sourceItem, side, nRows) {
let children = this._getVisibleChildren();
let index = children.indexOf(sourceItem.actor);
- if (index == -1) {
+ if (index == -1)
throw new Error('Item not found.');
- return;
- }
+
let pageIndex = Math.floor(index / this._childrenPerPage);
let pageOffset = pageIndex * this._childrenPerPage;